home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 2161 < prev    next >
Encoding:
Text File  |  1996-08-05  |  3.3 KB  |  102 lines
  1. Path: erich.triumf.ca!bennett
  2. From: bennett@erich.triumf.ca (P.Bennett)
  3. Newsgroups: comp.lang.c
  4. Subject: Re: Problem with c code, please help!
  5. Date: 19 Jan 1996 08:19 PST
  6. Organization: TRIUMF: Tri-University Meson Facility
  7. Distribution: world
  8. Message-ID: <19JAN199608194071@erich.triumf.ca>
  9. References: <surgsw-1901960148530001@128.206.206.86> <mikedorman-1901960725380001@205.148.200.150>
  10. NNTP-Posting-Host: erich.triumf.ca
  11. News-Software: VAX/VMS VNEWS 1.50    
  12.  
  13. In article <mikedorman-1901960725380001@205.148.200.150>, mikedorman@cedarnet.com (Mike Dorman) writes...
  14. >In article <surgsw-1901960148530001@128.206.206.86>,
  15. >surgsw@mizzou1.missouri.edu (Joel Weinstein) wrote:
  16. > 
  17. >> #include <stdio.h>
  18. >> main()
  19. >> {
  20. >>    int i=0;
  21. >>    char  word[100], c;
  22. > 
  23. >word is an array of 100 chars, there are no pointers declared here.
  24. >change word to this:
  25. >      char *word;
  26. >and it will be *much* easier.
  27.  
  28. But it won't work.  You now have reserved space for a pointer-to-char, but have
  29. not reserved any space for it to point to, not have you set it to point
  30. anywhere meaningful.
  31.  
  32. >>    printf("Enter a word:   ");
  33. >>    while( (c = getchar()) != '\n')  {
  34. >>       *word = c;
  35. >here, replace *word (which is a pointer to something...you didn't specify
  36. >which char in word you wanted it to point to, so it could be pointing to
  37. >just about anything) with
  38. >        strcat(word, *c);
  39.  
  40. strcat() requires two strings - word is still an uninitialized pointer, so
  41. strcat() will write to some unkown location.  This probably won't compile,
  42. since "*c" is invalid - you probably mean "&c", but that's no good either,
  43. since "c" is a single char, not a 0 terminated string.
  44.  
  45.  
  46. >Also, when getting string information from the user, you probably
  47. >shouldn't use a char array (I know it seems to make more sense that way,
  48. >but soon it'll become clear that it doesn't), but you should use a string
  49. >pointer, like this:
  50. >   char *word;
  51. >Now, word is a pointer to a string, which has no specific or limited
  52. >length.  Then you can use the standard functions like strcpy and strcat to
  53. >work on string pointers.
  54.  
  55. No - word is an uninitialized pointer, and no space is reserved for it to point
  56. to - using it  as suggested can only lead to disaster, as you write over random
  57. locations in memory.
  58.  
  59. > 
  60. >To do it with an array, you need another variable to keep track of how
  61. >many characters you've already put into word, so you can index the array
  62. >correctly:
  63.  
  64. <some (surprisingly!) valid code snipped>
  65. > 
  66. >   puts("You entered: ");
  67. >   puts(*word[0]);
  68.  
  69. Most people would use:
  70.     puts(word);
  71. In this case the name of an array acts like a pointer to the first element of
  72. the array.
  73.  
  74. >}
  75. > 
  76. >But, if you did it with a string pointer, it's this much easier:
  77. > 
  78. >main()
  79. >{ 
  80. >   char *word;
  81.  
  82. Where does "word" point??
  83.  
  84. > 
  85. >   puts("Enter a word: ");
  86. >   while(scanf("%s", word) != 0)
  87. This will store the input in some random location (perhaps over-writing some of
  88. the program or operating system  :-(  )
  89.  
  90.  
  91.  
  92. Peter Bennett VE7CEI                | Vessels shall be deemed to be in sight
  93. Internet: bennett@triumf.ca         | of one another only when one can be
  94. Packet: ve7cei@ve7kit.#vanc.bc.ca   | observed visually from the other
  95. TRIUMF, Vancouver, B.C., Canada     |                          ColRegs 3(k)
  96. GPS and NMEA info and programs: ftp://sundae.triumf.ca/pub/peter/index.html
  97.  
  98.  
  99.  
  100.  
  101.  
  102.